Skip to content

Add an HTTP route to get the current Timestamp#2850

Merged
gefjon merged 1 commit into
masterfrom
phoebe/timestamp-route
Jun 9, 2025
Merged

Add an HTTP route to get the current Timestamp#2850
gefjon merged 1 commit into
masterfrom
phoebe/timestamp-route

Conversation

@gefjon

@gefjon gefjon commented Jun 9, 2025

Copy link
Copy Markdown
Contributor

Description of Changes

Under a new router, /unstable, as /unstable/timestamp.

Also add a smoketest that this route is reachable and returns a JSON-encoded Timestamp.

API and ABI breaking changes

Introduces a new HTTP route, but marks it unstable.

Expected complexity level and risk

Testing

  • Manual call with cURL.
  • Added smoketest for reachability and well-typedness. No validation for the value.

Under a new router, `/unstable`, as `/unstable/timestamp`.

Also add a smoketest that this route is reachable and returns a JSON-encoded `Timestamp`.
@gefjon gefjon requested review from aasoni and cloutiertyler June 9, 2025 15:29

@cloutiertyler cloutiertyler left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM

@bfops bfops added the release-any Can land in any release window. Will not block a release deployment. label Jun 9, 2025
@gefjon gefjon added this pull request to the merge queue Jun 9, 2025
Merged via the queue into master with commit 0866fd3 Jun 9, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-any Can land in any release window. Will not block a release deployment.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants